Skip to content

[CADB-19] Allow users to search by PNC ID#113

Merged
kruncher merged 2 commits into
team/CADB-3from
dev/CADB-19
Jun 16, 2026
Merged

[CADB-19] Allow users to search by PNC ID#113
kruncher merged 2 commits into
team/CADB-3from
dev/CADB-19

Conversation

@kruncher

Copy link
Copy Markdown

Jira link

See CADB-19

Change description

This PR makes the following changes:

  • Amend RAML schema of event to include the existing property pncIdentifier.
  • Amend existing Jolt transformer such that pncIdentifier is mapped to the elastic search index.

Testing done

Unit tests have been amended to accomodate the Jolt transformer amendment.

Security Vulnerability Assessment

CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?

  • Yes
  • No

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@kruncher kruncher requested a review from a team as a code owner June 15, 2026 18:10
@kruncher kruncher requested review from NirmalaDeviDharmaraj, jyothi-balla and thomast1906 and removed request for a team June 15, 2026 18:10
@kruncher kruncher changed the base branch from main to team/CADB-3 June 15, 2026 18:17
@cpp-github-management

Copy link
Copy Markdown

Passed

@balanagarian balanagarian left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the team branch POM with unique Snapshot version using the ticket name.

@cpp-github-management

Copy link
Copy Markdown

Passed

@kruncher kruncher enabled auto-merge June 16, 2026 09:19
@kruncher kruncher merged commit 413b16c into team/CADB-3 Jun 16, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants